Skip to content

Conversation

@jkiddo
Copy link
Collaborator

@jkiddo jkiddo commented May 5, 2025

No description provided.

@jkiddo jkiddo mentioned this pull request Sep 26, 2025
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 19 out of 19 changed files in this pull request and generated 8 comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

* ^contact.telecom[=].value = "[email protected]"
* ^experimental = false
* ^caseSensitive = false
* ^caseSensitive = true
Copy link

Copilot AI Nov 24, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The caseSensitive property for the NPU CodeSystem is being changed from false to true. This is a breaking change that could affect existing implementations. NPU codes should be validated to ensure they are consistently cased in all existing profiles, examples, and value sets. Please document this breaking change in the release notes.

Copilot uses AI. Check for mistakes.
Copy link

Copilot AI commented Nov 24, 2025

@jkiddo I've opened a new pull request, #194, to work on those changes. Once the pull request is ready, I'll request review from you.

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 19 out of 19 changed files in this pull request and generated no new comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

* ^slicing.discriminator.path = "identifier.system"
* ^slicing.rules = #open
* ^slicing.ordered = false
* ^slicing.description = "Slice based on the identifier.systemm value which allows for official qualifications to\nDeclaring the slices, and their cardinalities, to allow a KL-code and a SNOMED CT code"
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@jkiddo Note that Copilot actually found and fixed a spelling mistake in the change suggestion above that you missed: systemm -> system

@jacand jacand self-requested a review November 28, 2025 17:08
Jesper Langfeldt Hagen Haffgaard and others added 2 commits December 9, 2025 10:08
…iblity and link to example of diagnosis and ranks.
[(181-review-encounter-example-missing)] More descriptions of respons…
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 19 out of 19 changed files in this pull request and generated 7 comments.

Extension: DkCoreDocumentReferenceVersionID
Id: dk-core-documentreference-version-id-extension
Title: "HL7 Denmark core DocumentReference Version ID extension"
Description: "Extension containing information about the version of the DocumentReference for a specific standard. The version is included in the R5 version of the resource."
Copy link

Copilot AI Dec 9, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The extension is missing context definition. Extensions should define where they can be used via ^context.type and ^context.expression. For a DocumentReference version extension, add:

* ^context.type = http://hl7.org/fhir/extension-context-type#element
* ^context.expression = "DocumentReference"
Suggested change
Description: "Extension containing information about the version of the DocumentReference for a specific standard. The version is included in the R5 version of the resource."
Description: "Extension containing information about the version of the DocumentReference for a specific standard. The version is included in the R5 version of the resource."
* ^context.type = http://hl7.org/fhir/extension-context-type#element
* ^context.expression = "DocumentReference"

Copilot uses AI. Check for mistakes.
* extension[0].url = "http://hl7.dk/fhir/core/StructureDefinition/dk-core-planned-start-date"
* extension[=].valueDateTime = "2025-11-14T09:00:00.0000+01:00"
* extension[+].url = "http://hl7.dk/fhir/core/StructureDefinition/dk-core-planned-end-date"
* extension[=].valueDateTime = "2025-11-15"
Copy link

Copilot AI Dec 9, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

[nitpick] The planned end date uses a date-only value "2025-11-15" while the planned start date uses a full dateTime "2025-11-14T09:00:00.0000+01:00". For consistency and clarity, consider using the same precision (dateTime format) for both planned dates, especially since the extension is defined as valueDateTime.

Suggested change
* extension[=].valueDateTime = "2025-11-15"
* extension[=].valueDateTime = "2025-11-15T09:00:00.0000+01:00"

Copilot uses AI. Check for mistakes.
Copy link

Copilot AI commented Dec 9, 2025

@jkiddo I've opened a new pull request, #197, to work on those changes. Once the pull request is ready, I'll request review from you.

* Initial plan

* Fix typo: reponsibilities -> responsibilities

Co-authored-by: jkiddo <[email protected]>

---------

Co-authored-by: copilot-swe-agent[bot] <[email protected]>
Co-authored-by: jkiddo <[email protected]>
@jkiddo jkiddo marked this pull request as ready for review December 9, 2025 10:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

10 participants